Skip to content

misc: Add fine-grained stats for index connector stats tracking#12759

Closed
xiaoxmeng wants to merge 1 commit intofacebookincubator:mainfrom
xiaoxmeng:export-D71599185
Closed

misc: Add fine-grained stats for index connector stats tracking#12759
xiaoxmeng wants to merge 1 commit intofacebookincubator:mainfrom
xiaoxmeng:export-D71599185

Conversation

@xiaoxmeng
Copy link
Copy Markdown
Contributor

Summary:
Add runtime stats to track the wall/cpu times spent in different stages during index lookup processing and integrate
with velox operator stats
Add metrics to monitor index lookup and table scan batch processing time
distribution to ease monitoring an
online service

Differential Revision: D71599185

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 22, 2025
@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D71599185

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 22, 2025

Deploy Preview for meta-velox canceled.

Name Link
🔨 Latest commit 293ed80
🔍 Latest deploy log https://app.netlify.com/sites/meta-velox/deploys/67de6b76c5b0810008d070e0

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D71599185

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D71599185

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D71599185

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D71599185

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D71599185

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D71599185

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D71599185

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D71599185

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D71599185

…bookincubator#12759)

Summary:

Add runtime stats to track the wall/cpu times spent in different stages during index lookup processing and integrate
with velox operator stats
Add metrics to monitor index lookup and table scan batch processing time
distribution to ease monitoring an
online service

Reviewed By: wenqiwooo

Differential Revision: D71599185
@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request was exported from Phabricator. Differential Revision: D71599185

@facebook-github-bot
Copy link
Copy Markdown
Contributor

This pull request has been merged in 7b5bcb4.

@conbench-facebook
Copy link
Copy Markdown

Conbench analyzed the 1 benchmark run on commit 7b5bcb41.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants